Hello @RMeli! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file package/MDAnalysis/transformations/rotate.py:

Line 101:80: E501 line too long (83 > 79 characters)
Line 106:80: E501 line too long (88 > 79 characters)
Line 114:80: E501 line too long (88 > 79 characters)

Comment last updated at 2024-11-29 21:45:39 UTC

Codecov Report

Attention: Patch coverage is 96.49123% with 2 lines in your changes missing coverage. Please review.

Project coverage is 93.66%. Comparing base (441e2c6) to head (29e32db).
Report is 2 commits behind head on develop.

@@             Coverage Diff             @@
##           develop    #4809      +/-   ##
===========================================
- Coverage    93.68%   93.66%   -0.03%     
===========================================
  Files          177      189      +12     
  Lines        21742    22808    +1066     
  Branches      3055     3055              
===========================================
+ Hits         20369    21362     +993     
- Misses         927     1000      +73     
  Partials       446      446              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

which I recall you're doing at the end anyway

Thanks @marinegor for the review! Yes, I'll update the file in a separate PR, pooling some of the [fmt] PRs, the reason being that cleaning up history to only have two commits is just too tedious.